Skip to content

chore: khmer translation now uses busra font#752

Merged
mcdurdin merged 2 commits into
masterfrom
chore/busra_for_khmer
May 26, 2026
Merged

chore: khmer translation now uses busra font#752
mcdurdin merged 2 commits into
masterfrom
chore/busra_for_khmer

Conversation

@Meng-Heng
Copy link
Copy Markdown
Collaborator

This PR calls Busra font for Khmer translation (with Unicode range) on the website. Changing to this font allow easy skim reading, clearer and bolder characters, and readability. Please review.

Test-bot: skip

User Interfaces

URL: http://keyman.com.localhost:8053/km/keyboards/
Screenshot 2026-05-21 at 4 24 45 in the afternoon

Screenshot 2026-05-21 at 4 24 58 in the afternoon

URL: http://keyman.com.localhost:8053/km/downloads/
image

image

@Meng-Heng Meng-Heng added this to the A19S30 milestone May 21, 2026
@Meng-Heng Meng-Heng added this to Keyman May 21, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Keyman May 21, 2026
@keymanapp-test-bot
Copy link
Copy Markdown

User Test Results

Test specification and instructions

User tests are not required

Comment thread cdn/dev/css/template.css Outdated
Comment on lines +268 to +273
/* Khmer */
:lang(km) {
/* The browser will automatically use Busra for Khmer chars and Cabin or Sans-serif for others */
font-family: 'Busra','Cabin', sans-serif;
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. For maintainability, can we move this alphabetically before Lao (line 289)?

  2. I don't know CSS. Are Cabin and sans-serif already handled in line 173? (I note the other languages only reference one font for family, and not in quotes)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll try removing the two fonts to see if Busra is working properly or not. I added the Unicode range after font-family so this might fix it without having to specify three fonts.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've got the answer now. The Cabin and sans-serif fonts are there as a fallback for other characters that aren't Khmer.

Since I use lang(km) to recognize the km in the URL (http://keyman.com.localhost:8053/km/keyboards/?q=k&x=0&y=0) to trigger a font change, it changes the font for the entire web page.

Busra font:
Screenshot 2026-05-22 at 5 02 50 in the afternoon

keyman.com:
Screenshot 2026-05-22 at 5 03 17 in the afternoon

Whereas the other font-family properties are being used for .lang-example selectors, so it does not impact any other elements; therefore, one value for font-family is enough.

Copy link
Copy Markdown
Contributor

@darcywong00 darcywong00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mcdurdin mcdurdin merged commit 13cd87c into master May 26, 2026
5 checks passed
@mcdurdin mcdurdin deleted the chore/busra_for_khmer branch May 26, 2026 12:38
@github-project-automation github-project-automation Bot moved this from Todo to Done in Keyman May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants